Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update #7

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Update #7

wants to merge 3 commits into from

Conversation

Justman100
Copy link

No description provided.

@0x7d8
Copy link
Member

0x7d8 commented Nov 1, 2024

I would not recommend putting the button next to the download button since on stuff like quilt there are multiple downloads, also a text field to copy from may be better

@Justman100
Fix
bca17e9 
Replace button with text field and check for multiple downloads
@Justman100
Copy link
Author

I would not recommend putting the button next to the download button since on stuff like quilt there are multiple downloads, also a text field to copy from may be better

See latest commit...

@0x7d8
Copy link
Member

0x7d8 commented Nov 2, 2024

can you move the input outside of the download step, so above all steps and make the input full width + disabled?

@Justman100
Copy link
Author

can you move the input outside of the download step, so above all steps and make the input full width + disabled?

How? Isn't it part of the download step?

@0x7d8
Copy link
Member

0x7d8 commented Nov 3, 2024

the script runs every part of the installation, not just downloading

@Justman100
Copy link
Author

the script runs every part of the installation, not just downloading

It would look silly if the input was above the buttons, as it is larger

@Justman100
Copy link
Author

@0x7d8

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Justman100 @0x7d8